Skip to content

Added Crystal.gitignore#1518

Closed
rhysd wants to merge 1 commit intogithub:masterfrom
rhysd:crystal
Closed

Added Crystal.gitignore#1518
rhysd wants to merge 1 commit intogithub:masterfrom
rhysd:crystal

Conversation

@rhysd
Copy link
Contributor

@rhysd rhysd commented May 13, 2015

I added gitignore file for Crystal programming language.

It ignores compiled bitcode files, object files and a compiler's cache directory, which are generated automatically by a compiler.

@rhysd rhysd changed the title Add gitignore for Crystal Added Crystal.gitignore May 13, 2015
@rhysd
Copy link
Contributor Author

rhysd commented May 15, 2015

I'm sorry that I forgot showing a link to Crystal programming langugage.

http://crystal-lang.org/

@Jesterovskiy
Copy link

Approve this PR! =)

@arcresu
Copy link
Contributor

arcresu commented Jul 6, 2015

Is there any documentation which explains why these particular rules are a good idea? Is this language used by enough people to make it worth adding to this collection of most useful templates?

@Jesterovskiy
Copy link

@arcresu This is base template https://github.com/manastech/crystal/blob/119ccc89065165bc937b3417e5ac2af2bab6e056/src/compiler/crystal/tools/init/template/gitignore.ecr, but *.bc, *.o needed too.

"used by enough people"? Ok, what is measure? Stars/Fork/Watch on Github?

@arcresu
Copy link
Contributor

arcresu commented Jul 6, 2015

Is there any official documentation? There's no specific measure of popularity - I see that this language is in alpha though and only has a handful of repositories here. From that search page, the most popular few don't seem to include the other rules in their .gitignores, just the .crystal rule.

@Jesterovskiy
Copy link

@arcresu http://crystal-lang.org/docs/index.html
Agree with you, all files *.bc, *.o is located in .crystal dir. Yes, language is in alpha, if for adding template to this repo, Crystal must make stable version release - ok, remove this PR and return later =)

@arcresu
Copy link
Contributor

arcresu commented Jul 12, 2015

I'll close this in favour of #1589 since that PR is a little more comprehensive. Comment there if you have any changes to suggest. Thanks!

@arcresu arcresu closed this Jul 12, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants